[FEATURE] Ragged Encoding. NOT WORKING YET. Only for Intermediate presentations#2282
[FEATURE] Ragged Encoding. NOT WORKING YET. Only for Intermediate presentations#2282Paulo-Mansano wants to merge 1 commit intoapache:mainfrom
Conversation
…ormRaggedTest. Modified: TfUtils, ColumnEncoder, EncoderFactory, TransformCustomTest
|
Hi @Paulo-Mansano , Thanks for opening the PR! In specific we need to look at : Where the metadata is allocated, (we want to modify it to allocate the ragged array) and Where the metadata frame is populated with information. (we want it to be in the form of a ragged array) I hope this helps clarify things. |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #2282 +/- ##
============================================
- Coverage 72.95% 72.94% -0.01%
- Complexity 46086 46103 +17
============================================
Files 1479 1480 +1
Lines 172654 172744 +90
Branches 33796 33817 +21
============================================
+ Hits 125955 126007 +52
- Misses 37201 37233 +32
- Partials 9498 9504 +6 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
New classes: ColumnEncoderRagged, TransformDummySeparatedTest, TransformRaggedTest.
Modified: TfUtils, ColumnEncoder, EncoderFactory, TransformCustomTest
What changes were proposed?
ColumnEncoderRaggedin order to implement ragged array encoding.TransformDummySeparatedTest,TransformRaggedTest.TfUtils,ColumnEncoder,EncoderFactoryjust to add RAGGED as a encoding method.How was this tested?
Related Issues